Skip to content

Plotting updates#50

Merged
georgeholt1 merged 14 commits into
mainfrom
plotting_updates
Apr 1, 2026
Merged

Plotting updates#50
georgeholt1 merged 14 commits into
mainfrom
plotting_updates

Conversation

@kpentland
Copy link
Copy Markdown
Collaborator

@kpentland kpentland commented Oct 27, 2025

Just updating legacy plotting routines for solved equilibria from FreeGS.

Main changes:

  • diverted equilibria separatrix will always be plotted in solid red (small bug meant that some where being plotted with a dashed line when the value of psi_boundary was negative).
  • limited equilibria will now be plotted with a LCFS in solid red (to be consistent with the above), but also shown is the separatrix from the primary X-point in dashed black.
  • primary X-point is marked with a bold red cross (alongside existing X-points as standard red crosses).
  • some legend labels have been updated.
  • enabling calculation of Br and Bz derivatives (for use in generating snowflake divertor geometries).

More docstrings also added.

@kpentland kpentland added the enhancement New feature or request label Oct 27, 2025
@georgeholt1
Copy link
Copy Markdown
Contributor

Thanks @kpentland, some solid improvements here

I've added some comments as reviews, nothing major

Comment thread freegs4e/plotting.py Outdated
Comment thread freegs4e/plotting.py Outdated
Comment thread freegs4e/plotting.py Outdated
Comment thread freegs4e/plotting.py Outdated
Comment thread freegs4e/plotting.py Outdated
Comment thread freegs4e/gradshafranov.py Outdated
Comment thread freegs4e/gradshafranov.py Outdated
Comment thread freegs4e/gradshafranov.py Outdated
@kpentland
Copy link
Copy Markdown
Collaborator Author

Much appreciated @georgeholt1, all good to go!

@georgeholt1 georgeholt1 merged commit 638c022 into main Apr 1, 2026
1 check passed
@georgeholt1 georgeholt1 deleted the plotting_updates branch April 1, 2026 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants